Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Ports Bay12 Posibrains[Ask me before Merge.] #772

Open
wants to merge 8 commits into
base: master
Choose a base branch
from

Conversation

RemieRichards
Copy link
Contributor

Posibrains are MMIs that can be made, and when clicked in hand request a ghost to inhabit them. They work EVERYWHERE an MMI works, Borgs, AI, Mechs

@RemieRichards
Copy link
Contributor Author

Woopsie Commits were done on the Github editor, hence the tiny changes per commit. :)

@RemieRichards RemieRichards changed the title Ports Bay12 Posibrains Ports Bay12 Posibrains [WIP/NOMERGE] Jun 30, 2014
@RemieRichards
Copy link
Contributor Author

Just dealing with some HUD based runtimes atm.

…ation13 into PosibrainPort

Conflicts:
	code/modules/mob/living/carbon/brain/posibrain.dm
@RemieRichards RemieRichards changed the title Ports Bay12 Posibrains [WIP/NOMERGE] Ports Bay12 Posibrains Jun 30, 2014
@RemieRichards
Copy link
Contributor Author

Runtimes removed.

@ghost
Copy link

ghost commented Jun 30, 2014

This would be good for mommi/keeper/drone construction rather than a fabricator shitting them out.

@RemieRichards
Copy link
Contributor Author

That's true @paprka

@GunHog
Copy link
Contributor

GunHog commented Jul 1, 2014

Build Positronic brain. Build a drone frame from the protolathe / fabricator. Activate the brain and get a ghost. Shove brain into drone frame. You now have a little drone pet! Of course, you could make it into a borg, AI, or mech. It is nice to have options.

@RemieRichards
Copy link
Contributor Author

@GunHog that'll come with the little improvements to Drones and we go along.

@RemieRichards RemieRichards changed the title Ports Bay12 Posibrains Ports Bay12 Posibrains[Ask me before Merge.] Jul 1, 2014
@Dorsisdwarf
Copy link
Contributor

Does it make it very, very explicit in the text when you enter a posibrain that you know NOTHING your old character knew, absolutely, clean start, no you may not go secbot and just "happen" to randomly search the guy you knew kill you?

@RemieRichards
Copy link
Contributor Author

@Dorsisdwarf Good catch, I forgot about Wiping past lives with wipe_memory() etc.

@ghost
Copy link

ghost commented Jul 1, 2014

Poll it. We can discuss opinions on porting posibrains in a forum thread, not here. Or IRC.

@GunHog
Copy link
Contributor

GunHog commented Jul 4, 2014

Did you conflict yourself, RR?

@RemieRichards
Copy link
Contributor Author

yeah, it's why the PR has such an odd [Tag] on it, I knew it would do this so I prioritised the feature which originally had less bugs, in this case, Drones.

usr << "<span class='notice'>Something is there but you can't see it.</span>"
return

var/msg = "<span class='info'>*---------*\nThis is \icon[src] \a <EM>[src]</EM>!\n[desc]\n"
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Isn't \a redundant?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Posibrain names aren't all supportive of A or all supportive of AN so \a is fine there.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No no, I mean, just not having A, AN, or \a, because isn't [src] supposed to insert it automatically?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Uh, the only uses of A and AN are to correctly insert A, AN etc in the sentence when you don't know what it is, as in [src] or [exampleobj], I don't believe [src] or [exampleobj] or whatever automatically place A/AN because that reduces the uses of A and AN down to "this is \a bike" which will always be "this is a bike".

tl;dr I think you're wrong, and if you're not wrong then \a and \an are stupid.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yeah, I am wrong, sorry. I tested by removing \a from /atom/verb/examine(), and it said "That's the fire alarm.". So [src] automatically inserts "the" or "The", not "a" or "an".

@GunHog
Copy link
Contributor

GunHog commented Jul 8, 2014

Plz finish 👍

@RemieRichards
Copy link
Contributor Author

Note, MrPerson is fixing some stuff with mechs and He says he'll try and help me with the Mech-MMI runtimes me and @GunHog found.

@JJRcop
Copy link
Contributor

JJRcop commented Jul 23, 2014

@RemieRichards poke 👉

@GunHog
Copy link
Contributor

GunHog commented Jul 23, 2014

Yes, fix conflicts and merge plz (After the runtime fix)

@GunHog
Copy link
Contributor

GunHog commented Jul 25, 2014

If this method is not suitable, you may consider a special MMI (Machine-Machine Interface) that would accept pAIs. That would have little difference behavior-wise with cyborgs and AIs, but the concern would be using them as mech pilots. (Such as a Durand army slaved to a traitor with orders to murderbone) Positronic mech pilots could never be an antagonist, while a pAI could be one by proxy.

@RemieRichards
Copy link
Contributor Author

GunHog. a Special MMI is EXACTLY what a Posibrian is, It would run into exactly the same bugs, Probably MORE with it being a pAI as well.

@GunHog
Copy link
Contributor

GunHog commented Aug 4, 2014

RR Plz Finnish K Thx L0L

@GunHog
Copy link
Contributor

GunHog commented Aug 7, 2014

Agreed, this has the added bonus of making pAIs more desired by ghosted players. It is not a bad way to get AIs playing as well. Artyom does often suffer the lack of AI players during lowpop.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants